Does this conform to our checkstyle? I thought "protected" should be on the same line as the method definition.

interesting, it passed the checkstyle tests so looks like that's not enforced.
I used automated refactoring to make the change, so I have no qualms about changing it to be on the same line. Will check in

@frode-carlsen Could you fix the indentation of the pom.xml file change ? Then rebase, and squash. Thanks

ok, made some cosmetic changes to make it align same as plugins with above/below, but the whole pom.xml needs reformatting. It's a mix of tabs and spaces and improperly aligned child elements. Can do it here, but think perhaps it should be a different PR?

@frode-carlsen Just format to spaces here, but squash your commits too

how about this?

Rebased, and merged. Thanks for your contribution !